-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
remove Plausible Analytics metions from CE #4121
Conversation
fec178e
to
aaf191f
Compare
@@ -47,9 +47,11 @@ | |||
<% end %> | |||
<%= @inner_content %> | |||
|
|||
<p class="text-center text-gray-500 text-xs py-8"> | |||
© <%= DateTime.utc_now().year() %> Plausible Analytics. All rights reserved. | |||
</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
</p> | ||
<%= if ee?() do %> | ||
<p class="text-center text-gray-500 text-xs py-8"> | ||
© <%= DateTime.utc_now().year() %> Plausible Analytics. All rights reserved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -1,6 +1,6 @@ | |||
<div class="mt-12 w-full md:max-w-xl md:mx-auto bg-white dark:bg-gray-800 md:shadow-md md:rounded px-8 py-6"> | |||
<p class="text-gray-900 text-xl font-black dark:text-gray-100"> | |||
Welcome to Plausible Analytics! | |||
Welcome to Plausible! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
issuer: @issuer_name | ||
) | ||
issuer_name = Plausible.product_name() | ||
NimbleTOTP.otpauth_uri("#{issuer_name}:#{user.email}", user.totp_secret, issuer: issuer_name) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should be OK: https://g.co/gemini/share/3806579d87a0
The v2.1.0-rc users who created TOTP secrets with issuer=Plausible Analytics shouldn't encounter any issues with this change. The new TOTP secrets would be issued by Plausible CE.
@ruslandoga merge at will! |
Changes
This PR removes some of Plausible Analytics mentions from CE builds. There are still places where this mention is kept:
analytics/assets/js/app.js
Line 108 in 23a6431
analytics/lib/plausible_web/controllers/stats_controller.ex
Line 381 in 23a6431
analytics/lib/plausible_web/templates/layout/_footer.html.heex
Line 23 in 23a6431
analytics/lib/plausible_web/templates/page/index.html.eex
Line 6 in 23a6431
Tests
Changelog
Documentation
Dark mode